Skip to content

feat: add pre-commit check for merge conflict markers#1670

Merged
davidornelas11 merged 2 commits intodevfrom
pre-commit-merge-syntax
Apr 8, 2026
Merged

feat: add pre-commit check for merge conflict markers#1670
davidornelas11 merged 2 commits intodevfrom
pre-commit-merge-syntax

Conversation

@scott-williams-az
Copy link
Copy Markdown
Contributor

Description

After resolving merge conflicts with a large number of resolutions, We need to make sure we do not miss any resolutions. This has happened a few times, and we want to make sure we do not accidentally include a broken file

Write pre-commit hook to make sure staged files do not include

<<<<<<< HEAD
...
=======
...
>>>>>>> [Commit hash]

Links

@scott-williams-az scott-williams-az requested a review from a team as a code owner April 2, 2026 17:57
@asu-jenkins-devops
Copy link
Copy Markdown
Collaborator

Copy link
Copy Markdown
Contributor

@davidornelas11 davidornelas11 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

looks good and works when I tested it locally

@davidornelas11 davidornelas11 merged commit 0d41301 into dev Apr 8, 2026
1 check passed
@davidornelas11 davidornelas11 deleted the pre-commit-merge-syntax branch April 8, 2026 22:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants